Skip to content

chore(runway): cherry-pick fix: point DIGEST_API_URL to production endpoint cp-7.71.1 cp-7.72.0#28098

Merged
chloeYue merged 1 commit into
release/7.71.1from
runway-cherry-pick-7.71.1-1774885573
Mar 30, 2026
Merged

chore(runway): cherry-pick fix: point DIGEST_API_URL to production endpoint cp-7.71.1 cp-7.72.0#28098
chloeYue merged 1 commit into
release/7.71.1from
runway-cherry-pick-7.71.1-1774885573

Conversation

@runway-github
Copy link
Copy Markdown
Contributor

@runway-github runway-github Bot commented Mar 30, 2026

Description

Switches the hardcoded fallback value for DIGEST_API_URL in
AppConstants.ts from the dev endpoint
(digest.dev-api.cx.metamask.io) to the production endpoint
(digest.api.cx.metamask.io).

Also moves the @metamask/sdk-communication-layer import to the top of
the file to follow alphabetical import ordering.

Changelog

CHANGELOG entry: null

Related issues

Fixes:

Manual testing steps

N/A

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the
    app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described
    in the ticket it closes and includes the necessary testing evidence such
    as recordings and or screenshots.

Made with Cursor


Note

Low Risk
Low risk: only changes a constant fallback URL and import ordering.
Main impact is that builds without DIGEST_API_URL will now call the
production digest service instead of the dev endpoint.

Overview
Updates AppConstants.DIGEST_API_URL to default to the production
https://digest.api.cx.metamask.io/api/v1 endpoint when the
DIGEST_API_URL env var is not set.

Also reorders the DEFAULT_SERVER_URL import to match expected import
ordering.

Written by Cursor
Bugbot
for commit
9d584a4. This will update automatically
on new commits. Configure
here.

[8603e1c](https://github.com/MetaMask/metamask-mobile/commit/8603e1ceacec64d79320da8377bed18061474cb6)

…dpoint (#28088)

## **Description**

<!-- 1. What is the reason for the change? The fallback URL for
`DIGEST_API_URL` was pointing at the development endpoint. -->
<!-- 2. What is the improvement/solution? Switch the hardcoded fallback
to the production endpoint so that builds without an explicit env var
hit production. -->

Switches the hardcoded fallback value for `DIGEST_API_URL` in
`AppConstants.ts` from the dev endpoint
(`digest.dev-api.cx.metamask.io`) to the production endpoint
(`digest.api.cx.metamask.io`).

Also moves the `@metamask/sdk-communication-layer` import to the top of
the file to follow alphabetical import ordering.

## **Changelog**

CHANGELOG entry: null

## **Related issues**

Fixes:

## **Manual testing steps**

N/A

## **Screenshots/Recordings**

### **Before**

N/A

### **After**

N/A

## **Pre-merge author checklist**

- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I've included tests if applicable
- [x] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.

<!-- Generated with the help of the pr-description AI skill -->

Made with [Cursor](https://cursor.com)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk: only changes a constant fallback URL and import ordering.
Main impact is that builds without `DIGEST_API_URL` will now call the
production digest service instead of the dev endpoint.
> 
> **Overview**
> Updates `AppConstants.DIGEST_API_URL` to default to the production
`https://digest.api.cx.metamask.io/api/v1` endpoint when the
`DIGEST_API_URL` env var is not set.
> 
> Also reorders the `DEFAULT_SERVER_URL` import to match expected import
ordering.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
9d584a4. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
@github-actions
Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-bots Bot team (for MetaMask Bot, Runway Bot, etc.) label Mar 30, 2026
Copy link
Copy Markdown
Contributor

@chloeYue chloeYue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions Bot added size-XS risk-low Low testing needed · Low bug introduction risk labels Mar 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 95%
click to see 🤖 AI reasoning details

E2E Test Selection:
The PR makes two changes to app/core/AppConstants.ts:

  1. Import reordering (cosmetic): Moving DEFAULT_SERVER_URL import to the top - no functional impact.

  2. DIGEST_API_URL fallback URL fix: Changed from https://digest.dev-api.cx.metamask.io/api/v1 (dev environment) to https://digest.api.cx.metamask.io/api/v1 (production environment). This is a bug fix correcting the default fallback URL from a dev endpoint to the correct production endpoint.

Investigation findings:

  • DIGEST_API_URL is only consumed in app/core/Engine/controllers/ai-digest-controller-init.ts where it's passed as baseUrl to AiDigestService for the AiDigestController
  • No E2E tests exist for AI Digest functionality
  • This change doesn't affect any user-facing flows covered by existing E2E test tags (accounts, confirmations, swaps, networks, etc.)
  • The change is purely a configuration/URL correction with no UI, navigation, or transaction flow impact
  • No performance implications from changing a URL string constant

No E2E test tags are warranted for this isolated configuration fix.

Performance Test Selection:
A URL string change in AppConstants has no performance impact. The AiDigestController URL change does not affect rendering, data loading, account lists, or any other performance-sensitive areas.

View GitHub Actions results

@sonarqubecloud
Copy link
Copy Markdown

@chloeYue chloeYue merged commit c0ae622 into release/7.71.1 Mar 30, 2026
59 checks passed
@chloeYue chloeYue deleted the runway-cherry-pick-7.71.1-1774885573 branch March 30, 2026 16:15
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

risk-low Low testing needed · Low bug introduction risk size-XS team-bots Bot team (for MetaMask Bot, Runway Bot, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants